Skip to content

Update dbt package reference #2092

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
elazarlachkar merged 1 commit into from
Jan 20, 2026
Merged

Update dbt package reference #2092

elazarlachkar merged 1 commit into from
Jan 20, 2026

Conversation

@elazarlachkar
Copy link
Contributor

@elazarlachkar elazarlachkar commented Jan 20, 2026 •
edited by coderabbitai bot
Loading

null

Summary by CodeRabbit

  • Chores
    • Updated data reliability monitoring dependencies to latest versions.

✏️ Tip: You can customize this high-level summary in your review settings.

@elazarlachkar elazarlachkar self-assigned this Jan 20, 2026
@linear
Copy link

linear bot commented Jan 20, 2026

CORE-261 Add identifier as comment to Cloud queries

@github-actions
Copy link
Contributor

github-actions bot commented Jan 20, 2026

👋 @elazarlachkar
Thank you for raising your pull request.
Please make sure to add tests and document all user-facing changes.
You can do this by editing the docs files in this pull request.

@coderabbitai
Copy link

coderabbitai bot commented Jan 20, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

Updated the git dependency revision for dbt-data-reliability from f5c062d to 9f64cbcaf across package manifest and lock file, maintaining consistency between dependency declarations.

Changes

Cohort / File(s) Summary
Dependency Version Update
elementary/monitor/dbt_project/package-lock.yml, elementary/monitor/dbt_project/packages.yml		 Bumped git dependency revision for dbt-data-reliability repository and updated corresponding sha1_hash in lock file to keep package definitions synchronized

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

🐰 A hop, skip, and git commit away,
Dependency versions dance and play!
From hash to hash, revision aligned,
Elementary updates, perfectly designed! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Update dbt package reference' accurately reflects the main change: updating the git dependency revision in both package-lock.yml and packages.yml for the dbt-data-reliability package.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot]
coderabbitai bot reviewed Jan 20, 2026
View reviewed changes
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents 
In `@elementary/monitor/dbt_project/package-lock.yml`:
- Around line 5-6: The sha1_hash key in package-lock.yml is not indented under
the git package entry and will be treated as a top-level YAML key; fix it by
indenting sha1_hash to the same level as revision (e.g., the same number of
spaces before sha1_hash as before revision) so both keys are siblings under the
git entry, ensuring consistent YAML indentation for revision and sha1_hash.

@elazarlachkar elazarlachkar merged commit 398b34a into master Jan 20, 2026
7 checks passed
@elazarlachkar elazarlachkar deleted the core-261-add-identifier-as-comment-to-cloud-queries branch January 20, 2026 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

@haritamar haritamar haritamar approved these changes

Assignees

@elazarlachkar elazarlachkar

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@elazarlachkar @haritamar